[flang-commits] [flang] [flang][driver] Remove `flang -help` tests (PR #89504)

Andrzej WarzyƄski via flang-commits flang-commits at lists.llvm.org
Mon Apr 22 02:41:39 PDT 2024


banach-space wrote:

Thanks for the feedback @tblah !

@Meinersbur and I discussed this one at EuroLLVM couple of weeks ago. I don't feel super strong about this one - I've actually sent it for review to see what the sentiments are :)

> They do make it easier to confirm my understanding of exactly what is being changed.

Well, I think that we should get into a habit of using "proper" tests for that :) In fact, I feel that folks assume that updating these files qualifies as "testing" (IMHO, it doesn't). That's one additional argument for removing them.

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


More information about the flang-commits mailing list