vitalybuka wrote:

> @vitalybuka Here's a more detailed explination on the motivations behind this 
> change from @amyw-msft, who is the original author of these changes on our 
> side. (And who I'll add as a co-author using fixup commits)
> 
> https://devblogs.microsoft.com/cppblog/msvc-address-sanitizer-one-dll-for-all-runtime-configurations/

Thanks. Would be possible to extract summary paragraph and put on top of the 
main  patch, with this URL.

Also, to submit, it would be nice to do with smaller patches with some time 
between them to let bots test them separately. Or it's going to be annoying to 
revert/reland such large patch.

Almost every item from description looks like a candidate for a patch.

Relaxing `// CHECK: #{{[1-3]}}` could be done in a separate patch.

https://github.com/llvm/llvm-project/pull/81677
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to