[llvm] [NewPM] Add pass options for `InternalizePass` to preserve GVs. (PR #91334)

Tim Besard via llvm-commits llvm-commits at lists.llvm.org
Tue May 14 04:31:41 PDT 2024


================
@@ -13,6 +13,11 @@
 ; -file and -list options should be merged, the apifile contains foo and j
 ; RUN: opt < %s -passes=internalize -internalize-public-api-list bar -internalize-public-api-file %S/apifile -S | FileCheck --check-prefix=FOO_J_AND_BAR %s
 
+; specifying through pass builder option
+; RUN: opt < %s -passes='internalize<preserve-gv=foo;preserve-gv=j>' -S | FileCheck --check-prefix=FOO_AND_J %s
+; RUN: opt < %s -passes='internalize<preserve-gv=foo;preserve-gv=bar>' -S | FileCheck --check-prefix=FOO_AND_BAR %s
+; RUN: opt < %s -passes='internalize<preserve-gv=foo;preserve-gv=j;preserve-gv=bar>' -S | FileCheck --check-prefix=FOO_J_AND_BAR %s
+
----------------
maleadt wrote:

Yeah, I had started this change using `,` before realizing the separator was already taken. Using other separators seemed more questionable than just repeating the flag.

https://github.com/llvm/llvm-project/pull/91334


More information about the llvm-commits mailing list