perf: add __NO_SIDE_EFFECTS__ comments (#9053)

chenfan0 · haoqunjiang · web-flow · commit d46df6bdb14b · 2024-03-13T21:48:45.000+08:00
Co-authored-by: Haoqun Jiang &lt;haoqunjiang@gmail.com&gt;
diff --git a/packages/runtime-core/src/devtools.ts b/packages/runtime-core/src/devtools.ts
@@ -123,6 +123,7 @@ export const devtoolsComponentRemoved = (
   }
 }
 
+/*! #__NO_SIDE_EFFECTS__ */
 function createDevtoolsComponentHook(hook: DevtoolsHooks) {
   return (component: ComponentInternalInstance) => {
     emit(
diff --git a/packages/server-renderer/src/helpers/ssrRenderAttrs.ts b/packages/server-renderer/src/helpers/ssrRenderAttrs.ts
@@ -17,7 +17,7 @@ import {
 } from '@vue/shared'
 
 // leading comma for empty string ""
-const shouldIgnoreProp = makeMap(
+const shouldIgnoreProp = /*#__PURE__*/ makeMap(
   `,key,ref,innerHTML,textContent,ref_key,ref_for`,
 )
 
diff --git a/packages/shared/src/makeMap.ts b/packages/shared/src/makeMap.ts
@@ -5,6 +5,8 @@
  * \/\*#\_\_PURE\_\_\*\/
  * So that rollup can tree-shake them if necessary.
  */
+
+/*! #__NO_SIDE_EFFECTS__ */
 export function makeMap(
   str: string,
   expectsLowerCase?: boolean,

Original file line number	Diff line number	Diff line change
`@@ -123,6 +123,7 @@ export const devtoolsComponentRemoved = (`
`123`	`123`	`}`
`124`	`124`	`}`
`125`	`125`
	`126`	`+/! #__NO_SIDE_EFFECTS__ /`
`126`	`127`	`function createDevtoolsComponentHook(hook: DevtoolsHooks) {`
`127`	`128`	`return (component: ComponentInternalInstance) => {`
`128`	`129`	`emit(`
Original file line number	Diff line number	Diff line change
`@@ -17,7 +17,7 @@ import {`
`17`	`17`	`} from '@vue/shared'`
`18`	`18`
`19`	`19`	`// leading comma for empty string ""`
`20`		`-const shouldIgnoreProp = makeMap(`
	`20`	`+const shouldIgnoreProp = /#__PURE__/ makeMap(`
`21`	`21`	`,key,ref,innerHTML,textContent,ref_key,ref_for`,
`22`	`22`	`)`
`23`	`23`