njames93 added a comment. In D128337#3602694 <https://reviews.llvm.org/D128337#3602694>, @aaron.ballman wrote:
> I like the changes -- this is a much nicer syntax for specifying > configuration options! > >> The only observable differences are support for the new syntax and >> -dump=config will emit using the new syntax. > > Do you expect the behavior of `-dump` to cause any problems for folks using > that option from a script? I can't think of any that aren't super contrived, > but maybe you've got more thoughts there. I wouldn't expect people are using the dump-config option for any scripts, It's only really there for debugging. > (Note, you should probably rebase your patch as it doesn't seem to apply > cleaning, so there's no precommit CI happening for it.) That's your fault :) https://github.com/llvm/llvm-project/commit/bb297024fad2f6c3ccaaa6a5c3a270f73f15f3ac Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D128337/new/ https://reviews.llvm.org/D128337 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
